Skip to content

acc: Minor fixes in script in restricted-execution#3126

Merged
denik merged 2 commits intomainfrom
denik/fix-script
Jun 26, 2025
Merged

acc: Minor fixes in script in restricted-execution#3126
denik merged 2 commits intomainfrom
denik/fix-script

Conversation

@denik
Copy link
Copy Markdown
Contributor

@denik denik commented Jun 26, 2025

  • The second summary command does not return json, so passing it to jq is confusing (jq also does not fail on empty input).
  • Use musterr to show that error is intentional.

The second summary command does not return json, so passing it to jq is
confusing (jq also does not fail on empty input).

Use musterr to show that error is intentional.

export DATABRICKS_BUNDLE_RESTRICTED_CODE_EXECUTION=1
title "With DATABRICKS_BUNDLE_RESTRICTED_CODE_EXECUTION=1, no envs are accessible"
trace errcode uv run $UV_ARGS -q $CLI bundle summary | jq '.experimental'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We run with pipefail.

@denik denik temporarily deployed to test-trigger-is June 26, 2025 07:28 — with GitHub Actions Inactive
@denik denik enabled auto-merge June 26, 2025 07:29
@denik denik added this pull request to the merge queue Jun 26, 2025
Merged via the queue into main with commit 2d935b6 Jun 26, 2025
21 of 23 checks passed
@denik denik deleted the denik/fix-script branch June 26, 2025 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants